Skip to content

Expose private fields in ActiveAnimation#24153

Merged
alice-i-cecile merged 1 commit intobevyengine:mainfrom
Hilpogar:root-motion-part-2
May 6, 2026
Merged

Expose private fields in ActiveAnimation#24153
alice-i-cecile merged 1 commit intobevyengine:mainfrom
Hilpogar:root-motion-part-2

Conversation

@Hilpogar
Copy link
Copy Markdown
Contributor

@Hilpogar Hilpogar commented May 6, 2026

Objective

Expose last_seek_time and just_completed in ActiveAnimation. This allows detecting loops in animations and compute deltas. This PR implements the second point proposed in #23355.

Solution

Add last_seek_time and just_completed methods in ActiveAnimation.

Testing

Regular CI

@mnmaita mnmaita added C-Feature A new feature, making something new possible D-Trivial Nice and easy! A great choice to get started with Bevy C-Usability A targeted quality-of-life change that makes Bevy easier to use A-Animation Make things move and change over time S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels May 6, 2026
@github-project-automation github-project-automation Bot moved this to Needs SME Triage in Animation May 6, 2026
@alice-i-cecile alice-i-cecile added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels May 6, 2026
@alice-i-cecile alice-i-cecile added this pull request to the merge queue May 6, 2026
Merged via the queue into bevyengine:main with commit 424fc92 May 6, 2026
51 checks passed
@github-project-automation github-project-automation Bot moved this from Needs SME Triage to Done in Animation May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Animation Make things move and change over time C-Feature A new feature, making something new possible C-Usability A targeted quality-of-life change that makes Bevy easier to use D-Trivial Nice and easy! A great choice to get started with Bevy S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants